Skip to content

ECC-8262: adopt OAuth mirror workflow pattern#3

Merged
pisto merged 2 commits into
masterfrom
ECC-8262
Jun 5, 2026
Merged

ECC-8262: adopt OAuth mirror workflow pattern#3
pisto merged 2 commits into
masterfrom
ECC-8262

Conversation

@pisto
Copy link
Copy Markdown
Contributor

@pisto pisto commented May 26, 2026

Align mirror-repository.yaml with the canonical pattern from terraform-cloud-infra:

  • OAuth token via BITBUCKET_CLIENT_ID/BITBUCKET_CLIENT_SECRET instead of static BITBUCKET_ACCESS_TOKEN
  • Daily schedule cron
  • Concurrency group to cancel redundant runs
  • permissions: contents: read

Copilot AI review requested due to automatic review settings May 26, 2026 17:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repository mirroring GitHub Actions workflow to follow the “OAuth mirror” pattern (Bitbucket OAuth client credentials + scheduled runs + concurrency + reduced GITHUB_TOKEN permissions).

Changes:

  • Add a daily scheduled trigger (cron: "0 0 * * *") alongside push-triggered runs.
  • Add a concurrency group to cancel in-progress redundant workflow runs.
  • Replace use of a static Bitbucket access token with an OAuth client-credentials flow and use the fetched token for the mirror target URL.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +41 to +43
- uses: Corvina-R-D/corvina-github-actions/mirror-repository@v5
with:
target: https://x-token-auth:${{ secrets.BITBUCKET_ACCESS_TOKEN }}@bitbucket.org/exorint/mirror-corvina-github-public-actions.git No newline at end of file
target: https://x-token-auth:${{ steps.bitbucket-auth.outputs.token }}@bitbucket.org/exorint/mirror-corvina-github-public-actions.git
@pisto pisto merged commit 2eee599 into master Jun 5, 2026
1 check failed
@pisto pisto deleted the ECC-8262 branch June 5, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants